messageTray: Remove an extraneous show

This means that the close button also does not have to be public.

diff --git a/js/ui/messageTray.js b/js/ui/messageTray.js
index 7b61d11b61bdc4e91297a0d87794e9055009751c..250c112958ca56020b600103e8c6c66fdf210f16 100644 (file)
--- a/js/ui/messageTray.js
+++ b/js/ui/messageTray.js
@@ -1406,12 +1406,12 @@ const SummaryItem = new Lang.Class({
         this.notificationStackView.add_actor(this.notificationStack);
         this.notificationStackWidget.add_actor(this.notificationStackView);
 
-        this.closeButton = Util.makeCloseButton();
-            this.closeButton.connect('clicked', Lang.bind(this, function() {
+        this._closeButton = Util.makeCloseButton();
+        this._closeButton.connect('clicked', Lang.bind(this, function() {
             source.destroy();
             source.emit('done-displaying-content');
         }));
-        this.notificationStackWidget.add_actor(this.closeButton);
+        this.notificationStackWidget.add_actor(this._closeButton);
         this._stackedNotifications = [];
 
         this._oldMaxScrollAdjustment = 0;
@@ -2666,12 +2666,8 @@ const MessageTray = new Lang.Class({
         if (this._clickedSummaryItemMouseButton == 1) {
             // Acknowledge all our notifications
             summaryItem.source.notifications.forEach(function(n) { n.acknowledged = true; });
-
-            child = summaryItem.notificationStackWidget;
-
-            let closeButton = summaryItem.closeButton;
-            closeButton.show();
             summaryItem.prepareNotificationStackForShowing();
+            child = summaryItem.notificationStackWidget;
         } else if (this._clickedSummaryItemMouseButton == 3) {
             child = summaryItem.rightClickMenu;
         }